Merge multiple if statements #1120
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
vcs-diff-lint found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
|
|
||
| return None |
Check warning
Code scanning / vcs-diff-lint
Trailing whitespace
|
|
||
| return None | ||
|
|
Check warning
Code scanning / vcs-diff-lint
Trailing whitespace
| try_count += 1 | ||
| log.error('Downloading error %s: %s (retrying...)', url, str(err)) | ||
| time.sleep(2 ** try_count) | ||
| except Exception as err: |
Check warning
Code scanning / vcs-diff-lint
FileDownloader.get: Catching too general exception Exception
| except requests.exceptions.RequestException as err: | ||
| try_count += 1 | ||
| log.error('Downloading error %s: %s (retrying...)', url, str(err)) | ||
| time.sleep(2 ** try_count) |
Check warning
Code scanning / vcs-diff-lint
FileDownloader.get: Undefined variable 'time'
Combine multiple if statements to reduce Duplicate code and improve readability